On a un groupe pour le CA aussi, dans le morceau "association" ca pourrait ptet etre pertinent de les lister ?

Peut-être oui. @Eskimon, tu fais parti du CA je crois, non ? Tu peux demander sur leur forum si ça leur convient ? Je vais aussi envoyer un mail à l'équipe de com' pour leur demander (même si ils diront sûrement oui, donc pour les avertir :D).

J'ai mis à jour ma PR ! Maintenant ça liste aussi les membres du CA et il y a des liens pour envoyer un mp à tous les membres du groupe.

J'ai ajouté une définition pour les groupes qui n'en avaient pas. J'ai écris de la merde donc si quelqu'un est apte à définir les rôles de chaque groupe, ce serait cool :D

Bon, si personne n'as rien à dire, on laisse les textes comme ça ? Ready for QA ?

Désolé @pierre-24 mais je n'ai pas encore implémenté les remarques de @Vayel faites sur le forum https://zestedesavoir.com/forums/sujet/3915/amelioration-de-la-page-de-contact/?page=1#p70669. J'essaye de faire ça ce soir ! ;)

Voilà c'est prêt à être QA ! D'ailleurs, toutes l'équipe de com' est d'accord pour être affichée sur cette page.

Rapport de QA: presque OK

• Pour une raison que je pige pas bien, le lien pour envoyer vers le forum de dev (forum_feedback_user) ne fonctionne pas. C'est surement une erreur de mon coté, parce que je vois pas de fautes de frappe/
• Pour la page de l'association, ça vaudrait la peine de dire "plus d'info sur la page dédiée" et de renvoyer sur /pages/association/, non ?

Pas grand chose, quoi :)

Voilà, c'est fixé !

COME BACK !

Quelques remarques que j'avais déjà fait fut une époque : cette page n'est absolument pas modulable à moins d'ajouter un groupe en dur dans le code. Pourquoi ne pas stocker ces informations dans le settings.py ?

Quelque chose comme ça ?

ZDS_APP['contact'] = [
    {
        'group': 'devs',
        'description': 'foo bar'
    },
    {
        'group': 'staff',
        ...
    }
    ...
]

Ça me semble plus propre. Un ordre peut aussi être ajouté pour pas que ça sorte en aléatoire dans le template.

Ça me semble plus propre. Un ordre peut aussi être ajouté pour pas que ça sorte en aléatoire dans le template.

Ah oui, pas bête ! Avec l'option de mettre -1 ou False pour ne pas l'afficher ou quelque chose comme ça.

Bon, je suis arrivé à ça :

ZDS_APP['contact'] = {
    'staff': {
        'position': 1,
        'title': u'Le Staff',
        'description': u"Le staff est constitué de certains membres du site dont le but est de contrôler le contenu "
                       u"publié sur {{ site_name }}. Ils sont en charge de la modération des messages sur les forums "
                       u"et commentaires, ainsi que de la validation et publication d'articles et/ou de tutoriels de "
                       u"{{ site_name }}" \
                           .format(site_name=ZDS_APP['site']['litteral_name'])
    },
    'com': {
        'position': 2,
        'title': u'Equipe de communication',
        'description': u"L'équipe de communication gère la communication du site sur les réseaux sociaux et sur la page "
                       u"d'accueil (notamment la partie \"A la Une\"). Elle a pour but de mettre en avant le contenu et "
                       u"les créations des membres."
    },
    'dev': {
        'position': 3,
        'title': u'Equipe technique',
        'description': u"L'équipe technique est constituée de certains membres du site dont le but est d'une part de "
                       u"s'assurer que le site reste toujours disponible en ligne, et d'autre part de corriger les "
                       u"bogues rencontrés sur le site ainsi que d'ajouter de nouvelles fonctionnalités.\n"
                       u"Des administrateurs systèmes, jusqu'aux designeurs, en passant par les développeurs back-end "
                       u"et intégrateurs front-end, ils s'occupent aussi de la maintenance du "
                       u"<a href='{{ repository }}'>dépôt officiel du projet</a>. Pour rester cohérent et organisé, le "
                       u"projet est orchestré par un Directeur Technique Canidé [DTC] et un Chef de Projet [CdP]"
                       u"(chaque développeur reste néanmoins autonome)." \
                           .format(repository=ZDS_APP['site']['repository'])
    },
    'asso': {
        'position': 4,
        'title': u"L'association",
        'description': u"L'association sert à assurer la pérénité du site au niveau financier et est gardienne des "
                       u"valeurs du site. Son conseil d'administration (dont les membres sont listés dessous) sert "
                       u"d'arbitre en cas de désaccord sur les nouvelles fonctionnalités du site. Pour plus "
                       u"d'informations, vous pouvez consulter "
                       u"<a href='{{ assoc_page }}'>la page dédiée à l'association</a> !" \
                           .format(assoc_page=reverse('zds.pages.views.association'))
    }
}

sauf qu'il y a la fonction reverse qui fait elle appel à settings.py donc impossible de mettre ça dans le fichier settings.py sinon on a une boucle !

Humm, je ne vois pas de solution "propre" pour ce problème. Tu as essayé quand même et tu as eu une erreur ?

La solution propre serait sans doute de surcharger Group avec une description et de faire une requête en base pour récupérer tous les groupes et les afficher dans la page. Votre proposition, elle demande de maintenir les groupes en base et les groupes dans le settings.py.

Du coup, les groupes en dur dans la page, ça fera très bien le taf.

Le 21 septembre 2015 17:26, Gérard Paligot notifications@github.com a
écrit :

La solution propre serait sans doute de surcharger Group avec une
description et de faire une requête en base pour récupérer tous les groupes
et les afficher dans la page. Votre proposition, elle demande de maintenir
les groupes en base et les groupes dans le settings.py.

—
Reply to this email directly or view it on GitHub
#2966 (comment)
.

C'est dommage mais ok.

Ben, à moins qu'on trouve une solution qui ne soit pas une usine à gaz…

2015-09-21 18:16 GMT+02:00 Gérard Paligot notifications@github.com:

C'est dommage mais ok.

—
Reply to this email directly or view it on GitHub
#2966 (comment)
.

Personnellement, j'aime bien l'idée du Proxy.

Oui enfin, ça reste la page de contact. Pas de sur-ingénérie, s'il vous plaît.

Ce n'était qu'une proposition.

La méthode Proxy est intéressante mais un peu lourde ici. Elle pourrait néanmoins nous être utile un jour si on veut rajouter des informations dans les groupes (pour une raison X ou Y).

@Situphen Du nouveau ?

Concernant l'idée de @firm1 de passer par un manager pour la requête en base de donnée, j'ai pas trouvé de moyen d'en créer un pour le modèle User.

Pour ce qui est de la variabilisation des groupes :

• dans settings.py, comme je l'ai dit, on ne peut pas si on garde des variables dans la description
• en passant par un modèle proxy, je pense qu'on aura le même problème, à moins d'interpréter ça dans la vue

Du coup je sais pas trop. D'un côté, avoir cette page variabilisée serait cool car plus simple pour ajouter/supprimer des groupes sur la page de contact. D'un autre côté, si on trouve pas de solution, pas la peine de faire une usine à gaz, c'est beaucoup plus simple et rapide de laisser ça dans les templates (vu le nombre de fois qu'on modifie cette page ^^).

Peut-être qu'il y a moyen de faire simple en utilisant un modèle proxy qui contiendrait un booléen pour afficher ou pas le groupe sur la page de contact et deux variables (description et contact) interprétée dans la vue. Je vais essayer de voir si ça peut fonctionner.

Bon, je viens de tester et on ne peut pas ajouter des champs au proxy. Est-ce que je crée une autre table héritant de Group mais n'étant pas un proxy ?

Si tu hérites, je pense que tu devrais t'inspirer de cette ressource (surtout les 2 dernières sections). Certes, c'est pour User mais je pense qu'il est aisé de l'adapter pour Group.

@Situphen Tu as eu le temps de tester ma ressource ou trouver une autre solution ?

@Situphen Tu as eu le temps de tester ma ressource ou trouver une autre solution ?

Il n'y a pas l'air d'avoir d'équivalent à AUTH_PROFILE_MODULE = 'accounts.UserProfile' et la méthode get_profile() de User pour Group. Je n'ai pas le temps de chercher plus.

@Situphen A première vue, c'est possible. Tu peux retrouver ici une PR qui fait ce changement. :)

sauf qu'il y a la fonction reverse qui fait elle appel à settings.py donc impossible de mettre ça dans le fichier settings.py sinon on a une boucle !

@Situphen : Tu as pensé à utiliser reverse_lazy (à importer) au lieu de reverse, ou je suis à côté de la plaque ?

@yapper-git : J'essayerai, merci !

On en est où ici ?

J'ai pas avancé faute de temps en ce moment. Je ferme, si quelqu'un veut s'en inspirer qu'il le fasse !